Skip to content

connected wallet info#47

Merged
phipsae merged 5 commits intoBuidlGuidl:mainfrom
nnennaokoye:feat/connected-wallet-info
May 29, 2025
Merged

connected wallet info#47
phipsae merged 5 commits intoBuidlGuidl:mainfrom
nnennaokoye:feat/connected-wallet-info

Conversation

@nnennaokoye
Copy link
Contributor

@nnennaokoye nnennaokoye commented May 27, 2025

Description

This PR adds visual indicators next to the wallet address in the header to display the Batch Membership Status and Check-in Status.

image

image

Additional Information

Related Issues

_Closes #4

Note: If your changes are small and straightforward, you may skip the creation of an issue beforehand and remove this section. However, for medium-to-large changes, it is recommended to have an open issue for discussion and approval prior to submitting a pull request.

Your ENS/address: 0x167142915AD0fAADD84d9741eC253B82aB8625cd

@vercel
Copy link

vercel bot commented May 27, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
batch16.buidlguidl.com ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 27, 2025 1:05pm

@nnennaokoye
Copy link
Contributor Author

@melanke @phipsae i created a new branch, i was getting errors from files i didn't work on, so i had to clone the repository again.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, it’s almost perfect!
There’s just a small problem:
The content is overflowing the page’s width. Now, with the addition of your component, the page has an unnecessary horizontal scrollbar.
Maybe this is happening because of the tooltips, but that’s just a guess.
You can try different ways to fix it, but I believe you could either tweak the tooltip position or move the badges to the left of the connect button.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed, please review

@melanke melanke requested a review from phipsae May 27, 2025 13:25
@phipsae
Copy link
Contributor

phipsae commented May 29, 2025

Thanks a lot for taking the issue! Let's merge

@melanke thanks for the review!

@phipsae phipsae merged commit 773470e into BuidlGuidl:main May 29, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants